Skip to content

Fix CI: Deal with a new clippy warning and pin a dependency which bumped its MSRV#968

Merged
mergify[bot] merged 2 commits intomasterfrom
fix-ci
Jan 1, 2026
Merged

Fix CI: Deal with a new clippy warning and pin a dependency which bumped its MSRV#968
mergify[bot] merged 2 commits intomasterfrom
fix-ci

Conversation

@psychon
Copy link
Owner

@psychon psychon commented Jan 1, 2026

No description provided.

error: called `unwrap` on `bytes_name` after checking its variant with `is_some`
   --> generator/src/generator/namespace/mod.rs:573:49
    |
572 |                     if bytes_name.is_some() && field_size == union_size {
    |                        -------------------- the check is happening here
573 |                         outln!(out, "Self({})", bytes_name.unwrap());
    |                                                 ^^^^^^^^^^^^^^^^^^^
    |
    = help: try using `match`
    = help: for further information visit https://rust-lang.github.io/rust-clippy/rust-1.92.0/index.html#unnecessary_unwrap
    = note: `-D clippy::unnecessary-unwrap` implied by `-D warnings`
    = help: to override `-D warnings` add `#[allow(clippy::unnecessary_unwrap)]`

Signed-off-by: Uli Schlachter <psychon@znc.in>
@mergify mergify bot merged commit 44780c3 into master Jan 1, 2026
21 checks passed
@mergify mergify bot deleted the fix-ci branch January 1, 2026 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants